Skip to content
This repository was archived by the owner on Jul 15, 2022. It is now read-only.

removed dict loading because api returns a list#44

Merged
sandromodarelli merged 1 commit intomasterfrom
hotfix/removing-loaddict-from-api_urls-endpoints
Mar 12, 2020
Merged

removed dict loading because api returns a list#44
sandromodarelli merged 1 commit intomasterfrom
hotfix/removing-loaddict-from-api_urls-endpoints

Conversation

@sandromodarelli
Copy link
Copy Markdown
Contributor

Did you labeled this PR?

Uncomment response

YES

What this PR does / why we need it:
fixing values returned by api urls update endpoints

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

fixing values returned by api urls update endpoints

@sandromodarelli sandromodarelli added the bug Something isn't working label Mar 12, 2020
@sandromodarelli sandromodarelli requested review from a user and prometherion March 12, 2020 10:05
Copy link
Copy Markdown
Contributor

@prometherion prometherion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sandromodarelli sandromodarelli merged commit c354867 into master Mar 12, 2020
@sandromodarelli sandromodarelli deleted the hotfix/removing-loaddict-from-api_urls-endpoints branch March 12, 2020 10:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants